Skip to content

feat(generate-transformation-test): honor explicit --from/--to range#141

Open
samcm wants to merge 1 commit into
masterfrom
feat/generate-test-explicit-range
Open

feat(generate-transformation-test): honor explicit --from/--to range#141
samcm wants to merge 1 commit into
masterfrom
feat/generate-test-explicit-range

Conversation

@samcm

@samcm samcm commented Jun 23, 2026

Copy link
Copy Markdown
Member

The --from/--to flags on generate-transformation-test were documented and parsed but never consumed — range discovery always anchored a duration-sized window at the head of available data. That makes it impossible to seed tests for sparse historical events (e.g. Rocket Pool minipool/megapool creations) which never fall in a recent window.

This applies an explicit window over the discovered range when supplied: numeric bounds map to block/slot/epoch strategies, non-numeric to time strategies, so mixed-axis models keep their heuristic value on the non-overridden axis.

The --from/--to flags were documented and parsed but never consumed; range
discovery always anchored a duration-sized window at the head of available
data. That makes it impossible to seed tests for sparse historical events
(e.g. Rocket Pool minipool/megapool creations) which never fall in a recent
window. Apply the explicit window over the discovered range when supplied:
numeric bounds map to block/slot/epoch strategies, others to time strategies,
so mixed-axis models keep their heuristic value on the non-overridden axis.
@samcm samcm requested a review from mattevans as a code owner June 23, 2026 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant